Skip to content

Conversation

@ShahabT
Copy link
Contributor

@ShahabT ShahabT commented Jun 10, 2025

What changed?

Server sends new wft in response of preious wft completion. This should not happen if the next wft is supposed to go to a different worker version.

Why?

When there is a transition, workflow could fall into an infinite loop of workflow tasks because they never go to the right worker.

How did you test it?

  • built
  • run locally and tested manually
  • covered by existing tests
  • added new unit test(s)
  • added new functional test(s)
    will be verified in Nightlies.

Potential risks

None

@ShahabT ShahabT requested a review from a team as a code owner June 10, 2025 00:13
@ShahabT ShahabT enabled auto-merge (squash) June 10, 2025 00:18
@ShahabT ShahabT merged commit c0e8b3e into main Jun 10, 2025
53 checks passed
@ShahabT ShahabT deleted the shahab/bypass branch June 10, 2025 00:40
Shivs11 pushed a commit that referenced this pull request Jun 10, 2025
## What changed?
Server sends new wft in response of preious wft completion. This should
not happen if the next wft is supposed to go to a different worker
version.

## Why?
When there is a transition, workflow could fall into an infinite loop of
workflow tasks because they never go to the right worker.

## How did you test it?
- [ ] built
- [ ] run locally and tested manually
- [ ] covered by existing tests
- [ ] added new unit test(s)
- [ ] added new functional test(s)
will be verified in Nightlies.

## Potential risks
None
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants